Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check remote user on install task #22

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Conversation

henrirosten
Copy link
Collaborator

  • On install task, warn if the remote user on the target host is not the current local user. The intention of this check is to make sure the user is aware that their remote login user is not the current local user, which usually means the nixos-anywhere login after installation will fail.
  • Provide a helper script add-remote-user.sh to make it easy to add the current local user as a remote user on the target host.

@henrirosten henrirosten marked this pull request as ready for review November 20, 2023 06:50
@henrirosten henrirosten requested review from joinemm, karim20230 and a team November 20, 2023 06:50
- On install task, warn if the remote user on the target host is not the
  current local user. The intention of this check is to make sure the
  user is aware that their remote login user is not the current local
  user, which usually means the nixos-anywhere login after installation
  will fail.
- Provide a helper script `add-remote-user.sh` to make it easy to add
  the current local user as a remote user on the target host.

Signed-off-by: Henri Rosten <[email protected]>
@henrirosten henrirosten force-pushed the check-remote-user-on-install branch from 294e0bb to 5cc6d22 Compare November 20, 2023 07:05
@henrirosten henrirosten merged commit bde9e05 into main Nov 20, 2023
1 check passed
@henrirosten henrirosten deleted the check-remote-user-on-install branch November 21, 2023 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants